Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change Stacked Area chart for Area chart for the Virtual Machine CPU States #18333

Merged
merged 2 commits into from
Jan 10, 2019

Conversation

PanSpagetka
Copy link
Contributor

@PanSpagetka PanSpagetka commented Jan 7, 2019

Change Stacked Area chart for Area chart for the Virtual Machine CPU States chart.
Now the values should not be aggregated.

Screenshots [Optional]

Before:
screenshot from 2019-01-07 13-54-55

After:
screenshot from 2019-01-07 13-57-19

Links [Optional]

https://bugzilla.redhat.com/show_bug.cgi?id=1527695

Steps for Testing/QA [Optional]

  1. Navigate to VM named CU data
  2. Monitoring -> Utilization
  3. Virtual Machine CPU States chart should be correct

@PanSpagetka
Copy link
Contributor Author

@h-kataria there are more Stacked Area charts, do you think I should also change them for the Area?

@h-kataria
Copy link
Contributor

@PanSpagetka if those are having the similar issue, then i would say let's change those as well.

@miq-bot
Copy link
Member

miq-bot commented Jan 10, 2019

Checked commits PanSpagetka/manageiq@24c0313~...0797f06 with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
0 files checked, 0 offenses detected
Everything looks fine. 🍪

@h-kataria h-kataria self-assigned this Jan 10, 2019
@h-kataria h-kataria added this to the Sprint 103 Ending Jan 21, 2019 milestone Jan 10, 2019
@h-kataria h-kataria merged commit b988aea into ManageIQ:master Jan 10, 2019
@h-kataria
Copy link
Contributor

@martinpovolny is there a label we use for charts related changes?

@martinpovolny
Copy link
Member

We have "charts" but only on the frontend.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants